Skip to content

Clarify username taken error message during registration#14

Open
TonyHeeee wants to merge 1 commit intoCMU-313:mainfrom
TonyHeeee:main
Open

Clarify username taken error message during registration#14
TonyHeeee wants to merge 1 commit intoCMU-313:mainfrom
TonyHeeee:main

Conversation

@TonyHeeee
Copy link
Copy Markdown

Summary

This PR clarifies the error message shown during user registration when a chosen username
conflicts with an existing user or group.

Previously, the client-side validation displayed a generic "username taken" message, which
did not clearly indicate that group name conflicts are also checked. I changed the error message to better inform users of this error. This change improves user feedback by making the error message more accurate and reduces user confusion.

Changes

  • Updated public/src/client/register.js to use a clearer translation key when a username
    or group name conflict is detected during registration.
  • Added a corresponding error message string in
    public/language/en-US/error.json.

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 21598710021

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 78.841%

Totals Coverage Status
Change from base Build 21503398961: 0.0%
Covered Lines: 25358
Relevant Lines: 30328

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants